Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SCID types to ChannelDetails #444

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Conversation

tnull
Copy link
Collaborator

@tnull tnull commented Jan 24, 2025

Closes #419.

We add the previously-omitted short_channel_id, inbound_scid_alias, outbound_scid_alias types to ChannelDetails.

@tnull tnull added this to the 0.5 milestone Jan 24, 2025
@tnull tnull requested a review from arik-so January 24, 2025 12:53
We add the previously-omitted `short_channel_id`, `inbound_scid_alias`,
`outbound_scid_alias` types to `ChannelDetails`.
@tnull
Copy link
Collaborator Author

tnull commented Jan 27, 2025

Kicked CI.

/// counterparty will recognize the alias provided here in place of the [`short_channel_id`]
/// when they see a payment to be routed to us.
///
/// Our counterparty may choose to rotate this value at any time, though will always recognize
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: "though will" reads like it's missing a subject

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It isn’t though? Btw, kinda preexisting as I stole the docs from LDK.

@tnull tnull merged commit 0558d91 into lightningdevkit:main Jan 27, 2025
7 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose short_channel_id in ChannelDetails
2 participants